Skip to content

Update orghierarchy#1224

Open
voneiden wants to merge 3 commits into
mainfrom
update-orghierarchy
Open

Update orghierarchy#1224
voneiden wants to merge 3 commits into
mainfrom
update-orghierarchy

Conversation

@voneiden

@voneiden voneiden commented Jun 25, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Adjusted package resolution settings to keep selected Helsinki-related packages available during installs and updates, improving environment consistency.

@coderabbitai

coderabbitai Bot commented Jun 25, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 7f39411e-f686-4160-ae17-500164f906c8

📥 Commits

Reviewing files that changed from the base of the PR and between ccf7e25 and e7bbe30.

⛔ Files ignored due to path filters (1)
  • uv.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • pyproject.toml
📝 Walkthrough

Walkthrough

Adds a new tool.uv.exclude-newer-package section to pyproject.toml and sets six HKI package entries to false.

Changes

UV configuration

Layer / File(s) Summary
Package-specific exclude-newer entries
pyproject.toml		 Adds a new tool.uv.exclude-newer-package section with six package names set to false.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A tiny hop in pyproject.toml,
Six little flags now sit so small.
🐰 I twitch my nose and give a cheer,
The config path is crystal clear.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title is relevant and concise, but it only names one affected package while the change updates uv settings for several packages.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch update-orghierarchy

Comment @coderabbitai help to get the list of available commands.

@azure-pipelines

azure-pipelines Bot commented Jun 25, 2026

Copy link
Copy Markdown

LINKEDEVENTS-API branch is deployed to platta: https://linkedevents-pr1224.api.dev.hel.ninja 🚀🚀🚀

voneiden added 2 commits June 25, 2026 15:25
@voneiden
deps: remove exclude newer from HKI packages
d6d47ff
@voneiden
deps: update django-orghierarchy for CSP compatibility
e7bbe30 
Updated version of django-orghierarchy moves inline styles into a file
which makes future CSP enforcement easier.

Refs: RATYK-207
@voneiden voneiden force-pushed the update-orghierarchy branch from fad0183 to e7bbe30 Compare June 25, 2026 12:39
@voneiden voneiden marked this pull request as ready for review June 25, 2026 12:39
@voneiden voneiden requested a review from a team as a code owner June 25, 2026 12:39
@sonarqubecloud

sonarqubecloud Bot commented Jun 25, 2026

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@azure-pipelines

azure-pipelines Bot commented Jun 25, 2026

Copy link
Copy Markdown

LINKEDEVENTS-API branch is deployed to platta: https://linkedevents-pr1224.api.dev.hel.ninja 🚀🚀🚀

1 similar comment
@azure-pipelines

azure-pipelines Bot commented Jul 8, 2026

Copy link
Copy Markdown

LINKEDEVENTS-API branch is deployed to platta: https://linkedevents-pr1224.api.dev.hel.ninja 🚀🚀🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@danipran danipran danipran approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@voneiden @danipran